<html>
<head><meta charset="utf-8"><title>reconsider caching on ci · clippy · Zulip Chat Archive</title></head>
<h2>Stream: <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/index.html">clippy</a></h2>
<h3>Topic: <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/reconsider.20caching.20on.20ci.html">reconsider caching on ci</a></h3>

<hr>

<base href="https://rust-lang.zulipchat.com">

<head><link href="https://rust-lang.github.io/zulip_archive/style.css" rel="stylesheet"></head>

<a name="228674517"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/reconsider%20caching%20on%20ci/near/228674517" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> matthiaskrgr <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/reconsider.20caching.20on.20ci.html#228674517">(Mar 03 2021 at 21:47)</a>:</h4>
<p>IIRC we disabled caching on CI because we would always build with latest rustc-master which would change multiple times a day anyway.<br>
Now that we pin to a nightly which only changes every 2 weeks, I wonder if it pays out to cache deps again....</p>



<a name="228675661"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/reconsider%20caching%20on%20ci/near/228675661" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> matthiaskrgr <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/reconsider.20caching.20on.20ci.html#228675661">(Mar 03 2021 at 21:56)</a>:</h4>
<p>Hm, apparently caching was removed when we started  pinning to a nightly, lol...<br>
<a href="https://github.com/rust-lang/rust-clippy/pull/6401/commits">https://github.com/rust-lang/rust-clippy/pull/6401/commits</a></p>



<a name="228735784"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/reconsider%20caching%20on%20ci/near/228735784" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> flip1995 <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/reconsider.20caching.20on.20ci.html#228735784">(Mar 04 2021 at 08:43)</a>:</h4>
<p>My intention was to always build Clippy from scratch, so we know that a clean build will succeed. This avoids 2 potential problems:</p>
<ol>
<li>Clippy only builds because the right things are cached.</li>
<li>CI fails because the wrong things are cached (and the only way to resolve this in GHA currently is to change the cache key with a version number or similar)</li>
</ol>
<p>And building Clippy doesn't take that long. Usually 1-2min. I more open to cache build artifacts for the dev tools. But that will result in us not being able to share a target dir between clippy_dev and clippy.</p>



<hr><p>Last updated: Aug 07 2021 at 22:04 UTC</p>
</html>